Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RIGHT and CENTER alignment in tabs #193

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

RIGHT and CENTER alignment in tabs #193

wants to merge 3 commits into from

Conversation

ctabin
Copy link

@ctabin ctabin commented Mar 17, 2017

Hello,

In order to solve issue #191, I did some enhancements and refactoring in tabulation support.
Let me know if this is valuable to merge :-) Any feedback is welcomed.

Best regards,
Cédric

Cédric Tabin added 3 commits March 17, 2017 21:37
Instead of directly generating a Chunk to do the left alignment (other ones
seemed to not work at all), the tab information is simply stored and used
when the according text chunk is generated.

See issue #191.
@ctabin ctabin changed the title RIGHT and CENTER alignment RIGHT and CENTER alignment in tabs Mar 18, 2017
@angelozerr
Copy link
Member

Thanks @ctabin for your PR. It's hard for me to find time to review it and your code changes a lot.

Just find time to review it, sorry -(

@ctabin
Copy link
Author

ctabin commented May 27, 2017

Hello @angelozerr,

No problem, just tell me if you need some explanations about my changes.

Best,
Cédric

@uded
Copy link

uded commented Oct 8, 2018

OK, is to be reviewed or closed? I am interested as I can't even go through those changes in an easy fashion. Maybe should be just closed?

@angelozerr
Copy link
Member

@uded close it and redo a PR please.

@ctabin
Copy link
Author

ctabin commented Oct 8, 2018

Hello @uded and @angelozerr and thanks for coming back. It makes already a long ago that I did this PR when I was testing the library to improve our PDF generation handling.

And yes, at the time, a bunch of changes were required in order to solve the problem of alignment which seems to be still an open issue. Maybe you have another idea on how to fix it ? Anyway, I'm still open to help, if any need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants